Remove Run 1-3 and disable Phase 2 RelVal workflows that read existing non-RAW files#50954
Conversation
|
cms-bot internal usage |
|
hold Until |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50954/49364 |
|
Pull request has been put on hold by @makortel |
|
A new Pull Request was created by @makortel for master. It involves the following packages:
@AdrianoDee, @DickyChant, @antoniovagnerini, @cmsbuild, @kfjack, @miquork, @sroychow can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
@cms-sw/pdmv-l2 If any of the Run 1-3 specific workflows are worth of keeping for Phase 2, we could also remove them now instead of commenting them out if you'd prefer. |
|
@cmsbuild, please test with cms-sw/cms-bot#2753 This PR should be testable after disabling few workflows on the cms-bot side (not very useful in the end, but let's demonstrate nevertheless). |
|
+1 Size: This PR adds an extra 16KB to repository Comparison Summary |
|
@cms-sw/pdmv-l2 Do you have any comments? |
makortel
force-pushed
the
evoIntegrationRelVal
branch
from
e584277 to
8aaeeed
Compare
|
See cms-sw/cms-bot#2753 that is used temporarily for testing of #50968 with this PR and #50953, and cms-sw/cms-bot#2765 for an update that is intended to be merged. |
|
+pdmv
|
|
Milestone for this pull request has been moved to CMSSW_20_0_X. Please open a backport if it should also go in to CMSSW_17_0_X. |
makortel
changed the title
|
unhold
|
|
This pull request is fully signed and it will be integrated in one of the next master IBs after it passes the integration tests. This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @mandrenguyen, @ftenchini (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
|
Please test |
|
-1 Failed Tests: RelVals Failed RelValsValueError: Undefined workflows: 10224.0, 250202.181, 25202.0 |
4 participants
PR description:
This PR is part of integrating the changes from the
EVOLUTION_Xbranch to themasterbranch in CMSSW_20_0_X. It removes Run 1-3 RelVal workflows that read existing non-RAW files that will not be redable in 20_0_X and onwards. Similar Phase 2 workflows are disabled by commenting them out, because they will likely be enabled in the future after their non-RAW input files have been remade.Following the strategy agreed in ORP 2026-05-05, it is opened already now to allow it to be reviewed such that it could be merged quickly when the
masterbranch moves to 20_0_X.Note unusual integration: This PR will be merged in 20_0_X regardless of the review status unless major flaws come up during the review and they have not been addressed.
Resolves cms-sw/framework-team#2226
PR validation:
None beyond the tests of EVOLUTION_X IBs